feat: Implemented Filters UI for Asset Events View#54640
feat: Implemented Filters UI for Asset Events View#54640bbovenzi merged 23 commits intoapache:mainfrom
Conversation
There was a problem hiding this comment.
Thanks for the PR, the direction looks good! I've left some comments on it. Also, I think there are still two things needed in this PR:
-
need to add labels for
start dateandend datefields -
make the layout more consistent with current filters, you could take a look at this one https://github.com/apache/airflow/pull/54049/files
-
(optional) reset filter feature is also needed (could be done in follow up one)
cc @RoyLee1224 I think you are more familiar with filter please help give it a quick look if you have time. Thanks!
airflow-core/src/airflow/ui/src/components/Assets/AssetEvents.tsx
Outdated
Show resolved
Hide resolved
airflow-core/src/airflow/ui/src/components/Assets/AssetEvents.tsx
Outdated
Show resolved
Hide resolved
airflow-core/src/airflow/ui/src/components/Assets/AssetEvents.tsx
Outdated
Show resolved
Hide resolved
|
Thanks @guan404ming for the review! |
71c9a9a to
5b18a59
Compare
pierrejeambrun
left a comment
There was a problem hiding this comment.
Similarly to other filtering options, can you handle the reset filter button too?
airflow-core/src/airflow/ui/src/components/Assets/AssetEvents.tsx
Outdated
Show resolved
Hide resolved
|
I've also implemented the reset button, similar to the counterpart in |
|
UI Looks good to me, nice fix 💪 |
|
Functionality looks good. Let's rebase to use the new components from #54895 |
bbovenzi
left a comment
There was a problem hiding this comment.
Actually functionality looks good. We can update the UX in another PR
* feat: implemented filter timestamp * refactor: minor * feat: remove key for the date input * feat: add run and dag filter * feat: add selector for run type * fix: fix translation for selector label * fix: add empty option in the selector * refactor: move filter variables to utils * fix: bug of total_entries * feat: add label * feat: beautify UI * feat: add label for searchbar * feat: use backend filter * feat: move filter components to AssetEventsFilter * feat: add reset button * fix: remove unused translation package * fix: remove unused translation package * feat: update reset ui * refactor: adjust the order of variables * refactor: move paramkeys to global * feat: conditionally render filter * refactor: rename showFilters --------- Co-authored-by: Brent Bovenzi <brent@astronomer.io> (cherry picked from commit 9385762)
* feat: implemented filter timestamp * refactor: minor * feat: remove key for the date input * feat: add run and dag filter * feat: add selector for run type * fix: fix translation for selector label * fix: add empty option in the selector * refactor: move filter variables to utils * fix: bug of total_entries * feat: add label * feat: beautify UI * feat: add label for searchbar * feat: use backend filter * feat: move filter components to AssetEventsFilter * feat: add reset button * fix: remove unused translation package * fix: remove unused translation package * feat: update reset ui * refactor: adjust the order of variables * refactor: move paramkeys to global * feat: conditionally render filter * refactor: rename showFilters --------- Co-authored-by: Brent Bovenzi <brent@astronomer.io>
* feat: implemented filter timestamp * refactor: minor * feat: remove key for the date input * feat: add run and dag filter * feat: add selector for run type * fix: fix translation for selector label * fix: add empty option in the selector * refactor: move filter variables to utils * fix: bug of total_entries * feat: add label * feat: beautify UI * feat: add label for searchbar * feat: use backend filter * feat: move filter components to AssetEventsFilter * feat: add reset button * fix: remove unused translation package * fix: remove unused translation package * feat: update reset ui * refactor: adjust the order of variables * refactor: move paramkeys to global * feat: conditionally render filter * refactor: rename showFilters --------- Co-authored-by: Brent Bovenzi <brent@astronomer.io>
* feat: implemented filter timestamp * refactor: minor * feat: remove key for the date input * feat: add run and dag filter * feat: add selector for run type * fix: fix translation for selector label * fix: add empty option in the selector * refactor: move filter variables to utils * fix: bug of total_entries * feat: add label * feat: beautify UI * feat: add label for searchbar * feat: use backend filter * feat: move filter components to AssetEventsFilter * feat: add reset button * fix: remove unused translation package * fix: remove unused translation package * feat: update reset ui * refactor: adjust the order of variables * refactor: move paramkeys to global * feat: conditionally render filter * refactor: rename showFilters --------- Co-authored-by: Brent Bovenzi <brent@astronomer.io>
* feat: implemented filter timestamp * refactor: minor * feat: remove key for the date input * feat: add run and dag filter * feat: add selector for run type * fix: fix translation for selector label * fix: add empty option in the selector * refactor: move filter variables to utils * fix: bug of total_entries * feat: add label * feat: beautify UI * feat: add label for searchbar * feat: use backend filter * feat: move filter components to AssetEventsFilter * feat: add reset button * fix: remove unused translation package * fix: remove unused translation package * feat: update reset ui * refactor: adjust the order of variables * refactor: move paramkeys to global * feat: conditionally render filter * refactor: rename showFilters --------- Co-authored-by: Brent Bovenzi <brent@astronomer.io>
* feat: implemented filter timestamp * refactor: minor * feat: remove key for the date input * feat: add run and dag filter * feat: add selector for run type * fix: fix translation for selector label * fix: add empty option in the selector * refactor: move filter variables to utils * fix: bug of total_entries * feat: add label * feat: beautify UI * feat: add label for searchbar * feat: use backend filter * feat: move filter components to AssetEventsFilter * feat: add reset button * fix: remove unused translation package * fix: remove unused translation package * feat: update reset ui * refactor: adjust the order of variables * refactor: move paramkeys to global * feat: conditionally render filter * refactor: rename showFilters --------- Co-authored-by: Brent Bovenzi <brent@astronomer.io>
* feat: implemented filter timestamp * refactor: minor * feat: remove key for the date input * feat: add run and dag filter * feat: add selector for run type * fix: fix translation for selector label * fix: add empty option in the selector * refactor: move filter variables to utils * fix: bug of total_entries * feat: add label * feat: beautify UI * feat: add label for searchbar * feat: use backend filter * feat: move filter components to AssetEventsFilter * feat: add reset button * fix: remove unused translation package * fix: remove unused translation package * feat: update reset ui * refactor: adjust the order of variables * refactor: move paramkeys to global * feat: conditionally render filter * refactor: rename showFilters --------- Co-authored-by: Brent Bovenzi <brent@astronomer.io>
* feat: implemented filter timestamp * refactor: minor * feat: remove key for the date input * feat: add run and dag filter * feat: add selector for run type * fix: fix translation for selector label * fix: add empty option in the selector * refactor: move filter variables to utils * fix: bug of total_entries * feat: add label * feat: beautify UI * feat: add label for searchbar * feat: use backend filter * feat: move filter components to AssetEventsFilter * feat: add reset button * fix: remove unused translation package * fix: remove unused translation package * feat: update reset ui * refactor: adjust the order of variables * refactor: move paramkeys to global * feat: conditionally render filter * refactor: rename showFilters --------- Co-authored-by: Brent Bovenzi <brent@astronomer.io>
* feat: implemented filter timestamp * refactor: minor * feat: remove key for the date input * feat: add run and dag filter * feat: add selector for run type * fix: fix translation for selector label * fix: add empty option in the selector * refactor: move filter variables to utils * fix: bug of total_entries * feat: add label * feat: beautify UI * feat: add label for searchbar * feat: use backend filter * feat: move filter components to AssetEventsFilter * feat: add reset button * fix: remove unused translation package * fix: remove unused translation package * feat: update reset ui * refactor: adjust the order of variables * refactor: move paramkeys to global * feat: conditionally render filter * refactor: rename showFilters --------- Co-authored-by: Brent Bovenzi <brent@astronomer.io>
* feat: implemented filter timestamp * refactor: minor * feat: remove key for the date input * feat: add run and dag filter * feat: add selector for run type * fix: fix translation for selector label * fix: add empty option in the selector * refactor: move filter variables to utils * fix: bug of total_entries * feat: add label * feat: beautify UI * feat: add label for searchbar * feat: use backend filter * feat: move filter components to AssetEventsFilter * feat: add reset button * fix: remove unused translation package * fix: remove unused translation package * feat: update reset ui * refactor: adjust the order of variables * refactor: move paramkeys to global * feat: conditionally render filter * refactor: rename showFilters --------- Co-authored-by: Brent Bovenzi <brent@astronomer.io>
* feat: implemented filter timestamp * refactor: minor * feat: remove key for the date input * feat: add run and dag filter * feat: add selector for run type * fix: fix translation for selector label * fix: add empty option in the selector * refactor: move filter variables to utils * fix: bug of total_entries * feat: add label * feat: beautify UI * feat: add label for searchbar * feat: use backend filter * feat: move filter components to AssetEventsFilter * feat: add reset button * fix: remove unused translation package * fix: remove unused translation package * feat: update reset ui * refactor: adjust the order of variables * refactor: move paramkeys to global * feat: conditionally render filter * refactor: rename showFilters --------- Co-authored-by: Brent Bovenzi <brent@astronomer.io>
* feat: implemented filter timestamp * refactor: minor * feat: remove key for the date input * feat: add run and dag filter * feat: add selector for run type * fix: fix translation for selector label * fix: add empty option in the selector * refactor: move filter variables to utils * fix: bug of total_entries * feat: add label * feat: beautify UI * feat: add label for searchbar * feat: use backend filter * feat: move filter components to AssetEventsFilter * feat: add reset button * fix: remove unused translation package * fix: remove unused translation package * feat: update reset ui * refactor: adjust the order of variables * refactor: move paramkeys to global * feat: conditionally render filter * refactor: rename showFilters --------- Co-authored-by: Brent Bovenzi <brent@astronomer.io>
* feat: implemented filter timestamp * refactor: minor * feat: remove key for the date input * feat: add run and dag filter * feat: add selector for run type * fix: fix translation for selector label * fix: add empty option in the selector * refactor: move filter variables to utils * fix: bug of total_entries * feat: add label * feat: beautify UI * feat: add label for searchbar * feat: use backend filter * feat: move filter components to AssetEventsFilter * feat: add reset button * fix: remove unused translation package * fix: remove unused translation package * feat: update reset ui * refactor: adjust the order of variables * refactor: move paramkeys to global * feat: conditionally render filter * refactor: rename showFilters --------- Co-authored-by: Brent Bovenzi <brent@astronomer.io>
* feat: implemented filter timestamp * refactor: minor * feat: remove key for the date input * feat: add run and dag filter * feat: add selector for run type * fix: fix translation for selector label * fix: add empty option in the selector * refactor: move filter variables to utils * fix: bug of total_entries * feat: add label * feat: beautify UI * feat: add label for searchbar * feat: use backend filter * feat: move filter components to AssetEventsFilter * feat: add reset button * fix: remove unused translation package * fix: remove unused translation package * feat: update reset ui * refactor: adjust the order of variables * refactor: move paramkeys to global * feat: conditionally render filter * refactor: rename showFilters --------- Co-authored-by: Brent Bovenzi <brent@astronomer.io>
* feat: implemented filter timestamp * refactor: minor * feat: remove key for the date input * feat: add run and dag filter * feat: add selector for run type * fix: fix translation for selector label * fix: add empty option in the selector * refactor: move filter variables to utils * fix: bug of total_entries * feat: add label * feat: beautify UI * feat: add label for searchbar * feat: use backend filter * feat: move filter components to AssetEventsFilter * feat: add reset button * fix: remove unused translation package * fix: remove unused translation package * feat: update reset ui * refactor: adjust the order of variables * refactor: move paramkeys to global * feat: conditionally render filter * refactor: rename showFilters --------- Co-authored-by: Brent Bovenzi <brent@astronomer.io>
* feat: implemented filter timestamp * refactor: minor * feat: remove key for the date input * feat: add run and dag filter * feat: add selector for run type * fix: fix translation for selector label * fix: add empty option in the selector * refactor: move filter variables to utils * fix: bug of total_entries * feat: add label * feat: beautify UI * feat: add label for searchbar * feat: use backend filter * feat: move filter components to AssetEventsFilter * feat: add reset button * fix: remove unused translation package * fix: remove unused translation package * feat: update reset ui * refactor: adjust the order of variables * refactor: move paramkeys to global * feat: conditionally render filter * refactor: rename showFilters --------- Co-authored-by: Brent Bovenzi <brent@astronomer.io>
* feat: implemented filter timestamp * refactor: minor * feat: remove key for the date input * feat: add run and dag filter * feat: add selector for run type * fix: fix translation for selector label * fix: add empty option in the selector * refactor: move filter variables to utils * fix: bug of total_entries * feat: add label * feat: beautify UI * feat: add label for searchbar * feat: use backend filter * feat: move filter components to AssetEventsFilter * feat: add reset button * fix: remove unused translation package * fix: remove unused translation package * feat: update reset ui * refactor: adjust the order of variables * refactor: move paramkeys to global * feat: conditionally render filter * refactor: rename showFilters --------- Co-authored-by: Brent Bovenzi <brent@astronomer.io>
* feat: implemented filter timestamp * refactor: minor * feat: remove key for the date input * feat: add run and dag filter * feat: add selector for run type * fix: fix translation for selector label * fix: add empty option in the selector * refactor: move filter variables to utils * fix: bug of total_entries * feat: add label * feat: beautify UI * feat: add label for searchbar * feat: use backend filter * feat: move filter components to AssetEventsFilter * feat: add reset button * fix: remove unused translation package * fix: remove unused translation package * feat: update reset ui * refactor: adjust the order of variables * refactor: move paramkeys to global * feat: conditionally render filter * refactor: rename showFilters --------- Co-authored-by: Brent Bovenzi <brent@astronomer.io>
* feat: implemented filter timestamp * refactor: minor * feat: remove key for the date input * feat: add run and dag filter * feat: add selector for run type * fix: fix translation for selector label * fix: add empty option in the selector * refactor: move filter variables to utils * fix: bug of total_entries * feat: add label * feat: beautify UI * feat: add label for searchbar * feat: use backend filter * feat: move filter components to AssetEventsFilter * feat: add reset button * fix: remove unused translation package * fix: remove unused translation package * feat: update reset ui * refactor: adjust the order of variables * refactor: move paramkeys to global * feat: conditionally render filter * refactor: rename showFilters --------- Co-authored-by: Brent Bovenzi <brent@astronomer.io>
* feat: implemented filter timestamp * refactor: minor * feat: remove key for the date input * feat: add run and dag filter * feat: add selector for run type * fix: fix translation for selector label * fix: add empty option in the selector * refactor: move filter variables to utils * fix: bug of total_entries * feat: add label * feat: beautify UI * feat: add label for searchbar * feat: use backend filter * feat: move filter components to AssetEventsFilter * feat: add reset button * fix: remove unused translation package * fix: remove unused translation package * feat: update reset ui * refactor: adjust the order of variables * refactor: move paramkeys to global * feat: conditionally render filter * refactor: rename showFilters --------- Co-authored-by: Brent Bovenzi <brent@astronomer.io>

Part of: #53053
This PR implements the filtering UI for viewing asset events. The filters include:
Timestamp -- date picker
DAG ID -- search bar with substring matching
Task ID -- search bar with substring matching
Demo:
https://github.com/user-attachments/assets/cfb496d2-fb84-4663-937c-7a61fa91526a